Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add DW profile check on every notification #172 #177

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

DaSpood
Copy link
Collaborator

@DaSpood DaSpood commented Apr 24, 2024

Until now the active profile was only checked for PROFILE_SWITCH notifications, so if for any reason the profile changed silently it would not be detected by the app. Now every notification will be checked, this may cause more switch requests than necessary but it should at least prevent getting locked out of the wanted profile until the next resume() call.

Until now the active profile was only checked for PROFILE_SWITCH
notifications, so if for any reason the profile changed silently it
would not be detected by the app. Now every notification will be
checked, this may cause more switch requests than necessary but it
should at least prevent getting locked out of the wanted profile until
the next resume() call.
@DaSpood DaSpood merged commit c82ed8f into v2_maintenance Apr 24, 2024
0 of 2 checks passed
@DaSpood DaSpood deleted the 172-datawedge-profile-swap-handling branch April 24, 2024 14:22
DaSpood added a commit that referenced this pull request Apr 24, 2024
Until now the active profile was only checked for PROFILE_SWITCH
notifications, so if for any reason the profile changed silently it
would not be detected by the app. Now every notification will be
checked, this may cause more switch requests than necessary but it
should at least prevent getting locked out of the wanted profile until
the next resume() call.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant